Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[devices]:Add ag9064 attributes for cpupld and swpld. #1516

Merged
merged 1 commit into from
Mar 23, 2018
Merged

[devices]:Add ag9064 attributes for cpupld and swpld. #1516

merged 1 commit into from
Mar 23, 2018

Conversation

StanleyCi
Copy link
Contributor

Signed-off-by: Stanley Chi stanley.chi@deltaww.com

- What I did

- How I did it

- How to verify it

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

Signed-off-by: Stanley Chi <stanley.chi@deltaww.com>
@lguohan lguohan merged commit baa9d45 into sonic-net:master Mar 23, 2018
abdosi added a commit that referenced this pull request Nov 21, 2020
756dd9c (HEAD -> 201911, origin/201911)
[201911 sonic-swss] Flushing FDB entries before removing BridgePort (#1516)
e3f22ea [201911/portsorch] Add
correct stat list for port buffer drop counters (#1509)

Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>
raphaelt-nvidia pushed a commit to raphaelt-nvidia/sonic-buildimage that referenced this pull request Jan 14, 2021
…onic-net#1516)

**What I did**
I added coded in sonic-swss orchagent to flush the FDB entries corresponding to the bridge port before removing the bridgeport.

**Why I did it**
I did it because if we don't flush the FDB entries before removing the bridge port and try to remove the bridge port, the bridge port removal results in a failure and the port operational status is set to down.

**How I verified it**
I created a VLAN interface, assigned IP address to it and added 2 Ethernet ports as tagged members to it. I waited for the switch to add its neighbors through these bridge ports. Then I removed VLAN membership of one of the bridge ports and see it is successful and the operational state of the port does not go down on removing the VLAN membership.

**Details if related**
flushFDBEntries() function is introduce in orchagent/fdborch.cpp and called from orchagent/portsorch.cpp

Co-authored-by: Madhan Babu <madhan@arc-build-server.mtr.labs.mlnx>
renukamanavalan added a commit to renukamanavalan/sonic-buildimage that referenced this pull request Mar 24, 2021
* 553936b (HEAD, origin/master, origin/HEAD, master) route_check: Fix hanging & logging level (sonic-net#1520)
* ed45412 [show][config] add support for setting and displaying switching modes on Y cable (sonic-net#1501)
* bf46638 Handling error scenario of adding port to Vlan which is part of LAG (sonic-net#1516)
* ae39883 Fix bug: show vlan config for vlan with no members (sonic-net#1503)
* 3a482ac [test] Update unit test coverage for command 'show mac' (sonic-net#1504)
* 4a0c010 [config] Disable/enable container monitoring when stopping/starting services (sonic-net#1499)
@ghost ghost mentioned this pull request Mar 24, 2021
4 tasks
lguohan pushed a commit that referenced this pull request Mar 24, 2021
* 553936b (HEAD, origin/master, origin/HEAD, master) route_check: Fix hanging & logging level (#1520)
* ed45412 [show][config] add support for setting and displaying switching modes on Y cable (#1501)
* bf46638 Handling error scenario of adding port to Vlan which is part of LAG (#1516)
* ae39883 Fix bug: show vlan config for vlan with no members (#1503)
* 3a482ac [test] Update unit test coverage for command 'show mac' (#1504)
* 4a0c010 [config] Disable/enable container monitoring when stopping/starting services (#1499)
yxieca added a commit to yxieca/sonic-buildimage that referenced this pull request Mar 26, 2021
utilities:
* 83f068b 2021-03-22 | Handling error scenario of adding port to Vlan which is part of LAG (sonic-net#1516) (HEAD -> 202012) [Sudharsan Dhamal Gopalarathnam]
* 470e8ce 2021-03-24 | Enable PFCWD only on ports where PFC is enabled (sonic-net#1508) [Andriy Yurkiv]
* 09ef2e0 2021-03-22 | [show][config] add support for setting and displaying switching modes on Y cable (sonic-net#1501) [vdahiya12]
* 0d17d37 2021-03-24 | Warmboot script improvements - timeout exec, disable swss autorestart, remove trap (sonic-net#1495) [Vaibhav Hemant Dixit]
* 2718cd8 2021-03-24 | [show] Fix int status of LAGs, configured as Vlan members (sonic-net#1478) [maksymbelei95]
* cc168fb 2021-03-22 | Fix bug: show vlan config for vlan with no members (sonic-net#1503) [allas-nvidia]

swss:
* 5d8d1fb 2021-03-26 | Revert "Revert "[buffermgr] Support maximum port headroom checking (sonic-net#1607)" (sonic-net#1675)" (sonic-net#1682) (HEAD -> 202012) [Prince Sunny]
* f8df1f8 2021-03-26 | [Dynamic Buffer Calc] Enhance the field checking in table handling (sonic-net#1680) [Stephen Sun]
* 6328c9f 2021-03-22 | [MuxOrch] FDB ageout safety check (sonic-net#1674) [Prince Sunny]
* e1d733e 2021-03-21 | reduce severity of log to info in case of flush on non-existing member (sonic-net#1669) [allas-nvidia]

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
lguohan pushed a commit that referenced this pull request Mar 27, 2021
utilities:
* 83f068b 2021-03-22 | Handling error scenario of adding port to Vlan which is part of LAG (#1516) (HEAD -> 202012) [Sudharsan Dhamal Gopalarathnam]
* 470e8ce 2021-03-24 | Enable PFCWD only on ports where PFC is enabled (#1508) [Andriy Yurkiv]
* 09ef2e0 2021-03-22 | [show][config] add support for setting and displaying switching modes on Y cable (#1501) [vdahiya12]
* 0d17d37 2021-03-24 | Warmboot script improvements - timeout exec, disable swss autorestart, remove trap (#1495) [Vaibhav Hemant Dixit]
* 2718cd8 2021-03-24 | [show] Fix int status of LAGs, configured as Vlan members (#1478) [maksymbelei95]
* cc168fb 2021-03-22 | Fix bug: show vlan config for vlan with no members (#1503) [allas-nvidia]

swss:
* 5d8d1fb 2021-03-26 | Revert "Revert "[buffermgr] Support maximum port headroom checking (#1607)" (#1675)" (#1682) (HEAD -> 202012) [Prince Sunny]
* f8df1f8 2021-03-26 | [Dynamic Buffer Calc] Enhance the field checking in table handling (#1680) [Stephen Sun]
* 6328c9f 2021-03-22 | [MuxOrch] FDB ageout safety check (#1674) [Prince Sunny]
* e1d733e 2021-03-21 | reduce severity of log to info in case of flush on non-existing member (#1669) [allas-nvidia]

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
stepanblyschak pushed a commit to stepanblyschak/sonic-buildimage that referenced this pull request May 10, 2021
…onic-net#1516)

*Handled error scenario when adding a port to Vlan which is already part of a LAG. Added unit test to cover the scenario. This is fix for the bug sonic-net#4456

Signed-off-by: Sudharsan Dhamal Gopalarathnam <sudharsand@nvidia.com>
raphaelt-nvidia pushed a commit to raphaelt-nvidia/sonic-buildimage that referenced this pull request May 23, 2021
* 553936b (HEAD, origin/master, origin/HEAD, master) route_check: Fix hanging & logging level (sonic-net#1520)
* ed45412 [show][config] add support for setting and displaying switching modes on Y cable (sonic-net#1501)
* bf46638 Handling error scenario of adding port to Vlan which is part of LAG (sonic-net#1516)
* ae39883 Fix bug: show vlan config for vlan with no members (sonic-net#1503)
* 3a482ac [test] Update unit test coverage for command 'show mac' (sonic-net#1504)
* 4a0c010 [config] Disable/enable container monitoring when stopping/starting services (sonic-net#1499)
carl-nokia pushed a commit to carl-nokia/sonic-buildimage that referenced this pull request Aug 7, 2021
* 553936b (HEAD, origin/master, origin/HEAD, master) route_check: Fix hanging & logging level (sonic-net#1520)
* ed45412 [show][config] add support for setting and displaying switching modes on Y cable (sonic-net#1501)
* bf46638 Handling error scenario of adding port to Vlan which is part of LAG (sonic-net#1516)
* ae39883 Fix bug: show vlan config for vlan with no members (sonic-net#1503)
* 3a482ac [test] Update unit test coverage for command 'show mac' (sonic-net#1504)
* 4a0c010 [config] Disable/enable container monitoring when stopping/starting services (sonic-net#1499)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants